feat: add USING heap to pg_cron installation script#2242
Conversation
PostgreSQL Package Dependency Analysis: PR #2242
SummaryNo packages had MAJOR version updates. Full Analysis ResultsPostgreSQL 15 Dependency ChangesExtracting PostgreSQL 15 dependencies...
Runtime Closure Size
Raw Dependency ClosurePostgreSQL 17 Dependency ChangesExtracting PostgreSQL 17 dependencies...
Runtime Closure Size
Raw Dependency Closure |
PostgreSQL Extension Dependency Analysis: PR #2242
SummaryNo extensions had dependencies with MAJOR version updates. Full Analysis ResultsPostgreSQL 15 Extension DependenciesPostgreSQL 17 Extension DependenciesOrioleDB 17 Extension Dependencies |
f5a07d1 to
7ebbd2e
Compare
This comment has been minimized.
This comment has been minimized.
7ebbd2e to
4e88e54
Compare
This comment has been minimized.
This comment has been minimized.
4e88e54 to
e7379b2
Compare
|
If we make this change to the targeted pg_cron tables, It's now pinned to heap forever. If orioledb later fixes the DELETE bug and you'd want these tables on orioledb, the install SQL hard-codes heap and an upgrade won't move them. Migrating an existing table off heap later requires an explicit ALTER TABLE ... SET ACCESS METHOD. I think we should research if anyone already applies TAM in our existing running projects before we make this change and release it. Possibly not, but I believe it is worth checking so that we don't have to fix it later if it turns out they are. |
e7379b2 to
584071b
Compare
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
584071b to
fd1ec60
Compare
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment has been minimized.
This comment has been minimized.
7ecc4a7 to
64b8ef0
Compare
64b8ef0 to
4c734c3
Compare
What kind of change does this PR introduce?
Fix
pg_cron's installation script and addUSING heapto create heap tables. Otherwise smoke tests fails for orioledb tests with the error:What is the current behavior?
ORI-172
What is the new behavior?
The installation script will have the following statement:
Additional context
Add any other context or screenshots.